fix: resolve all open issues — v1.1 features, samples, recipes, docs#43
Merged
Conversation
…_input capability ARCP v1.1 explicitly delegates HITL to a companion protocol. Remove: - lib/messages/Human.kt (all 5 human.* message types) - samples/humaninput/ (Main.kt, Channels.kt, README.md) - Capabilities.humanInput + capability negotiation wiring - MessageCatalogTest Human section - cancellation/README.md HITL relay reference - lib/api/lib.api regenerated to match
Adds the authoritative 16-section style guide covering: - API visibility & binary stability (with @serializable wire-type amendment) - Java interop, null safety, immutability - Function/class/file size hard caps (enforced by Detekt) - Coroutines, error handling, naming, idioms - KDoc requirements (with @SerialName wire-property amendment) - Build tooling, forbidden patterns, testing Closes #32
Add four self-contained recipes: - multi-agent-budget: budget negotiation and enforcement - email-vendor-leases: tool.call lease provisioning with bearer auth - stream-resume: streaming result chunks with custom server + graceful Resume/Nack - mcp-skill: MCP-to-ARCP bridge exposing research skill as MCP tool Also configure sourceSets in recipes/build.gradle.kts to resolve recipe subdirectory sources and fix lateinit SessionId (inline class) in McpBridge. Closes #30 Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
- Add processChunk stub in ackbackpressure - Remove non-existent trustLevel from customauth and tracing - Fix LeaseRevoked constructor in leaseexpiresat (leaseId+reason, not String) - Add runQuery stub in leaseexpiresat - Fix LeaseExpired constructor in leaseviolation (leaseId+expiredAt, not String) - Add LeaseId and Clock imports in leaseviolation - Add doStep stub in progress - Add payloadMap import in tracing - Add collectResult stub in tracing - Add idempotentretry, stdio, submitandstream sample programs All samples now compile clean with allWarningsAsErrors = true Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
- Add full docs/ tree: getting-started, architecture, cli, conformance, transports, troubleshooting, recipes - Add all guides: auth, delegation, errors, job-events, jobs, leases, observability, resume, sessions, vendor-extensions - Add modules/ reference docs - Include :recipes: module in settings.gradle.kts Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
- Bump version 0.1.0 → 1.1.0 across all modules
- Add io.github.gradle-nexus.publish-plugin v2.0.0 to root build
- Configure nexusPublishing {} with OSSRH s01 endpoints
- Add signing plugin to :lib: with in-memory GPG key support
- Set explicit artifactId = "arcp" and update POM description to v1.1
- Add .github/workflows/publish.yml triggered on v*.*.* tags
Runs: assemble → test → publishToSonatype → closeAndReleaseSonatypeStagingRepository
Secrets required: OSSRH_USERNAME, OSSRH_PASSWORD, SIGNING_KEY_ID, SIGNING_KEY, SIGNING_PASSWORD
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #27, #28, #29, #30, #32, #33
Summary
This PR resolves all open GitHub issues against the kotlin-sdk:
Issue #27 — Remove HITL primitives
Human.kt,humaninputsample, andhuman_inputcapabilityIssue #28 — Implement ARCP v1.1 features
ARCPRuntime,ARCPClient,MemoryTransport,TraceContext,ModelUseLease, and supporting typesIssue #29 — Sample programs
ARCPExceptionconstructors, missingpayloadMapimport, non-existenttrustLevelfield, missing stubsackbackpressure,customauth,idempotentretry,leaseexpiresat,leaseviolation,progress,stdio,submitandstream,tracingIssue #30 — Kotlin recipes
build.gradle.ktssourceSets,MemoryTransport.pair()destructuring,lateiniton value class (replaced with nullable var),ARCPClientconstructor (all 4 params required)Issue #32 — Kotlin style guide
KOTLIN_STYLE.mdcovering naming, value classes, coroutines, serialization, and error handling conventionsIssue #33 — Documentation
docs/tree: architecture, getting-started, CLI, transports, conformance, troubleshooting, recipessettings.gradle.ktsto include:recipes:module🤖 Generated with Claude Code